<html>
<head><meta charset="utf-8"><title>Need an MCP reviewer · t-compiler · Zulip Chat Archive</title></head>
<h2>Stream: <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/index.html">t-compiler</a></h2>
<h3>Topic: <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/Need.20an.20MCP.20reviewer.html">Need an MCP reviewer</a></h3>

<hr>

<base href="https://rust-lang.zulipchat.com">

<head><link href="https://rust-lang.github.io/zulip_archive/style.css" rel="stylesheet"></head>

<a name="247827848"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/Need%20an%20MCP%20reviewer/near/247827848" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Andy Wang <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/Need.20an.20MCP.20reviewer.html#247827848">(Jul 31 2021 at 23:01)</a>:</h4>
<p>I would like to make an MCP based on <a href="https://internals.rust-lang.org/t/x-py-remove-python-2-support/15045">https://internals.rust-lang.org/t/x-py-remove-python-2-support/15045</a> and the feedbacks. Would anyone on the compilers team like to be nominated as the reviewer?</p>



<a name="247843686"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/Need%20an%20MCP%20reviewer/near/247843686" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Joshua Nelson <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/Need.20an.20MCP.20reviewer.html#247843686">(Aug 01 2021 at 06:30)</a>:</h4>
<p><span class="user-mention silent" data-user-id="116122">simulacrum</span> would be a good reviewer (but I'm still skeptical this is a necessary change and I'd like to hear some more justification of why it's needed)</p>



<a name="247843694"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/Need%20an%20MCP%20reviewer/near/247843694" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Joshua Nelson <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/Need.20an.20MCP.20reviewer.html#247843694">(Aug 01 2021 at 06:31)</a>:</h4>
<p>This doesn't _really_ fit under T-compiler but bootstrap doesn't have its own team so T-compiler is probably as good a place as any</p>



<a name="247847604"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/Need%20an%20MCP%20reviewer/near/247847604" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Andy Wang <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/Need.20an.20MCP.20reviewer.html#247847604">(Aug 01 2021 at 08:25)</a>:</h4>
<p><span class="user-mention silent" data-user-id="232545">Joshua Nelson</span> <a href="#narrow/stream/131828-t-compiler/topic/Need.20an.20MCP.20reviewer/near/247843686">said</a>:</p>
<blockquote>
<p><span class="user-mention silent" data-user-id="116122">simulacrum</span> would be a good reviewer (but I'm still skeptical this is a necessary change and I'd like to hear some more justification of why it's needed)</p>
</blockquote>
<p>I won't mention anything about <code>x.py clean</code>, I agree now that unnecessary recompilations are bugs that need to be solved separately.</p>
<p>But I still think we shouldn't be stuck writing Python 2.7 for whatever changes we make to <code>bootstrap.py</code> though</p>



<a name="247854115"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/Need%20an%20MCP%20reviewer/near/247854115" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Léo Lanteri Thauvin <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/Need.20an.20MCP.20reviewer.html#247854115">(Aug 01 2021 at 11:14)</a>:</h4>
<p>Just know that hou are not <em>forced</em> to nominate a reviewer for the MCP. You can if you know one, but when you open the MCP, all the compiler team members and contributors will be pinged anyway</p>



<a name="247859889"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/Need%20an%20MCP%20reviewer/near/247859889" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> simulacrum <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/Need.20an.20MCP.20reviewer.html#247859889">(Aug 01 2021 at 13:43)</a>:</h4>
<p>Yes, I can likely be the assigned party, but I agree with <span class="user-mention" data-user-id="232545">@Joshua Nelson</span> that I don't think the time has come -- there's no real pain that I am aware of from supporting both, and <em>not</em> doing so doesn't seem to give us any real benefits (since binaries are still named differently etc on different platforms).</p>



<hr><p>Last updated: Aug 07 2021 at 22:04 UTC</p>
</html>